Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

emacs-lisp: Speed up semantic #9354

Closed
wants to merge 1 commit into from

Conversation

delaanthonio
Copy link
Contributor

Resolve #1907.

Emacs 25 sets semanticdb-find-default-throttle as a mode-local variable in
emacs-lisp-mode and includes the omniscient throttle which slows down indexing.
Let's override the mode-local variable so it doesn't include the omniscient
throttle.

See #7736 for additional context.

Resolve syl20bnr#1907.

Emacs 25 sets semanticdb-find-default-throttle as a mode-local variable in
emacs-lisp-mode and includes the omniscient throttle which slows down indexing.
Let's override the mode-local variable so it doesn't include the omniscient
throttle.

See syl20bnr#7736 for additional context.
@syl20bnr
Copy link
Owner

What do we loose when we deactivate this throttle ? I found this https://www.gnu.org/software/emacs/manual/html_node/semantic/Search-Throttle.html but it does not tell what feature are enabled.

@delaanthonio
Copy link
Contributor Author

I'm not exactly sure what we lose when we deactivate this throttle either. Although I've noticed that Semantic no longer parses several Elisp files when I'm spelling out a word in Elisp-mode.

@duianto
Copy link
Collaborator

duianto commented May 4, 2019

The PR:
Improve the use of semantic #11511
is now on the develop branch.

The author stated that this PR might have been made obsolete:

This pull request has the same effects as #9354, except that it works globally rather that only works in emacs-lisp-mode, and it's more clear what we have done and what we will lose.

@duianto duianto closed this May 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants